Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@packages/visual-editor/src/components/locator/locatorUtils.ts`:
- Around line 142-158: The parseMapStartingLocation function currently uses
parseFloat() which accepts malformed numeric strings like "40.7abc" instead of
requiring valid numeric input. Replace the parseFloat calls for latitude and
longitude with Number() instead, and update the validation checks from isNaN()
to Number.isFinite() to ensure the entire input string is numeric and not just a
numeric prefix. This will reject invalid coordinate strings while accepting
valid numeric values.

In `@packages/visual-editor/src/components/locator/LocatorWrapper.tsx`:
- Line 114: The call to `setSessionTrackingEnabled(true)` on the searcher object
is being executed synchronously during the render phase of the LocatorWrapper
component, which is a side effect that should not occur during render. Move this
call into a useEffect hook with an appropriate dependency array (likely
depending on the searcher instance) to ensure the side effect only runs after
the component mounts and when the searcher is initialized, not during the render
phase itself.

---

Nitpick comments:
In `@packages/visual-editor/src/components/locator/LocatorWrapper.tsx`:
- Around line 204-211: The handleDrag function is used as a dependency in
mapProps but is not memoized, causing mapProps to recalculate on every render.
Wrap the handleDrag function definition with useCallback hook and include its
dependencies (setMapCenter, setMapRadius, setShowSearchAreaButton) in the
useCallback dependency array. This will ensure that handleDrag maintains
referential equality across renders unless its dependencies change, preventing
unnecessary recalculations of mapProps that includes handleDrag in its
definition around lines 483-500.

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@packages/visual-editor/src/components/locator/Filters.tsx`:
- Around line 147-170: The hardcoded `openNowCheckBoxId` constant on line 147
creates duplicate DOM ids when multiple Filters component instances render on
the same page, breaking the label and input association. Since React.useId() is
not available in React 17.0.2, generate a unique identifier instead by using
useRef with a unique value (such as a timestamp or random string) initialized
once when the component mounts, then use this generated id value for both the
input id attribute and the label htmlFor attribute. This ensures each instance
gets a unique id without requiring a parent prop or React version upgrade.

In `@packages/visual-editor/src/components/locator/Map.tsx`:
- Around line 274-283: The coordinate parsing in the Map component is too
lenient with `parseFloat()` which accepts partial numeric strings like "40abc".
Replace the `parseFloat()` calls for both mapStartingLocation.latitude and
mapStartingLocation.longitude with `Number()` for strict parsing that rejects
partial numeric input. Additionally, replace the `isNaN()` validation checks for
both lat and lng with `Number.isFinite()` to properly reject both NaN and
Infinity values, ensuring the validation properly distinguishes between valid
finite numbers and invalid inputs.
- Around line 80-97: The DEFAULT_MAKI_ICON_NAME is assigned using the first item
from makiIconOptions array, but this array order is non-deterministic since it
derives from Object.entries(makiIconModules) which uses Vite's
import.meta.glob() without guaranteed ordering. To fix this, either sort the
makiIconEntries array before creating makiIconOptions to ensure consistent
ordering across all environments and builds, or explicitly select a specific
known icon name by searching through makiIconEntries or makiIconOptions instead
of relying on the first index position. Ensure the solution maintains the
current data structure while providing deterministic default selection.